Skip to content

docs java apm fix#70

Open
JsonBlack09 wants to merge 2 commits into
masterfrom
Docs-Java-APM-fix
Open

docs java apm fix#70
JsonBlack09 wants to merge 2 commits into
masterfrom
Docs-Java-APM-fix

Conversation

@JsonBlack09

Copy link
Copy Markdown

В инструкцию добавлены значения, для настройки из конфига newrelic.yml

@JsonBlack09 JsonBlack09 requested a review from Arsamoon March 17, 2025 11:49
Comment on lines +42 to +47
```yaml
log_file_path: stdout
license_key: '0123456789-123456789-123456789-123456789'
app_name: MY_AWESOME_APP
host: gmonit-collector.<DOMAIN>.ru
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а зачем

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Потому что ниже по документации, буквально в трех строчках ниже, именно такой формат записи. И документация, без этих строк, не позволяет установить инфра агент с помощью файла конфига, т.к. непонятно какие параметры указывать.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

при чем тут инфра агент?
в каких случаях вообще нужна установка с конфигом?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, ошибся, не инфра, а апм джава. В случаях, когда установка через переменные окружения невозможна.

@JsonBlack09 JsonBlack09 requested a review from krevedkokun April 21, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants